Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HTML5] Fix audio buffer size and latency hint. #43358

Merged
merged 1 commit into from
Nov 6, 2020

Conversation

Faless
Copy link
Collaborator

@Faless Faless commented Nov 6, 2020

The size of the audio buffer was incorrectly doubled when creating the script processor.
latencyHint is expressed in seconds, not milliseconds.
Additionally, on some browsers it actually affect the performance and stability of the audio driver.
For this reason it has been completely disabled (interactive) and a not has been left for future reference.

Fixes: #43210

The size of the audio buffer was incorrectly doubled when creating the
script processor.
latencyHint is expressed in seconds, not milliseconds.
Additionally, on some browsers it actually affect the performance and
stability of the audio driver.
For this reason it has been completely disabled (interactive) and a not
has been left for future reference.
@Faless Faless added bug platform:web topic:porting topic:audio regression cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Nov 6, 2020
@Faless Faless added this to the 4.0 milestone Nov 6, 2020
@akien-mga akien-mga merged commit 6797ab6 into godotengine:master Nov 6, 2020
@akien-mga
Copy link
Member

Thanks!

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Nov 11, 2020
@akien-mga
Copy link
Member

Cherry-pick included in #43454.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stuttering Audio on Chrome/Chromium (HTML5, Threads)
2 participants